[kotlin-server][jaxrs-spec] treat parameters with a default value as non-null#14413
Conversation
|
@jimschubert @dr4ke616 @karismann @Zomzog @andrewemery @4brunu @yutaka0m I'm pinging you because you're listed as kotlin maintainers. |
|
@jimschubert could you please have a look at this MR? It is a small change and non-breaking. |
|
rebased onto openapitools/master |
|
@wing328 hi I got no response from Kotlin maintainers for a month. Could you help getting this merged? The change is really quite simple: |
|
I think we need to consider |
|
@exaV sorry for the delay in merging this PR as there are too many PRs pending review. |
|
FYI. I've filed #16265 to fix the default value (e.g. |
Without this change parameters with a default value as described here https://swagger.io/docs/specification/describing-parameters/ would be treated as nullable. With this change they are now correctly marked as non-null.
PR checklist
This is important, as CI jobs will verify all generator outputs of your HEAD commit as it would merge with master.
These must match the expectations made by your contribution.
You may regenerate an individual generator by passing the relevant config(s) as an argument to the script, for example
./bin/generate-samples.sh bin/configs/java*.For Windows users, please run the script in Git BASH.
master(6.3.0) (minor release - breaking changes with fallbacks),7.0.x(breaking changes without fallbacks)